-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
build: update base and development dependencies #8
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Once DaCe 0.16 is released (it should be any moment) we should update, there is one small feature that we use.
However, now MyPy flags a lot of other stuff.
hooks: | ||
- id: mypy | ||
files: src|tests | ||
args: [--no-install-types] | ||
additional_dependencies: | ||
- pytest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You should add pytest again, it now complains about that in the test.
30c8c07
to
08436ff
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- There is still the pytest issue that was not there before.
- We should disable the TCH003 rule.
|
Pushed the wrong button, silly I know
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Add:
pyproject.toml
.Change:
pre-commit
config.dace
andjax
.